Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add YAML compatibility. #73

Closed
wants to merge 2 commits into from
Closed

Add YAML compatibility. #73

wants to merge 2 commits into from

Conversation

cjroth
Copy link

@cjroth cjroth commented Sep 14, 2013

No description provided.

@mashpie
Copy link
Owner

mashpie commented Sep 16, 2013

great, thanks! I'd like to merge, but need passing tests. Could you take some care to speed up?

@cjroth
Copy link
Author

cjroth commented Sep 16, 2013

Sure thing. Also, would it make sense if I add CSV support as well?

On Monday, September 16, 2013, Marcus Spiegel wrote:

great, thanks! I'd like to merge, but need passing tests. Could you take
some care to speed up?


Reply to this email directly or view it on GitHubhttps://github.com//pull/73#issuecomment-24498632
.

Chris Roth
Boulder, CO
cjroth.com http://www.cjroth.com

@mashpie
Copy link
Owner

mashpie commented Sep 16, 2013

thanks, csv is too limited I think. There are plans to support multi-plural forms which won't fit into a flat file.

and... sometimes there will be pluggable storage support, so that would be a place to put csv, redis, mongo, couch, etc. :)

@cjroth
Copy link
Author

cjroth commented Sep 16, 2013

Ah OK nice :) Reason I mentioned it was that my client was wondering if he
could export spreadsheets to CSV, but it sounds like that will be a much
more flexible solution then!

On Monday, September 16, 2013, Marcus Spiegel wrote:

thanks, csv is too limited I think. There are plans to support
multi-plural forms which won't fit into a flat file.

and... sometimes there will be pluggable storage support, so that would be
a place to put csv, redis, mongo, couch, etc. :)


Reply to this email directly or view it on GitHubhttps://github.com//pull/73#issuecomment-24505122
.

Chris Roth
Boulder, CO
cjroth.com http://www.cjroth.com

@mashpie mashpie added this to the backlog milestone Jan 31, 2016
@mashpie
Copy link
Owner

mashpie commented Feb 6, 2016

It's time to cleanup. I will probably merge #79 in near future. That PR adds pretty same features as yours but passes test. Don't worry too much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants